Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump go version to v1.23.1 #13204

Closed
wants to merge 3 commits into from

Conversation

kalpanathanneeru21
Copy link

DELETE THIS TEMPLATE BEFORE SUBMITTING

In order to have a good experience with our community, we recommend that you
read the contributing guidelines for making a PR, and understand the lifecycle
of a Packer PR:

https://github.com/hashicorp/packer/blob/master/.github/CONTRIBUTING.md#opening-an-pull-request

Describe the change you are making here!

Please include tests. Check out these examples:

If your PR resolves any open issue(s), please indicate them like this so they will be closed when your PR is merged:

Closes #xxx
Closes #xxx

@kalpanathanneeru21 kalpanathanneeru21 requested a review from a team as a code owner November 7, 2024 11:02
@lbajolet-hashicorp
Copy link
Contributor

Hi @kalpanathanneeru21,

Without justification, I do not understand why we should bump the minimum required version for building Packer to 1.23.1. This is the kind of change that will eventually happen later, when justified by incompatible changes in the codebase, but in the meantime we should leave it as-is.

Please note that while 1.21.0 is the minimum required version, Packer CLI is compiled and released with Go 1.22.9 for now, which might be updated if there is a good reason, independently from the minimum version specified in go.mod.

With that, while I thank you for the contribution, we will not accept this for now, so I'll close your PR, but please feel free to reopen if you think I was too quick to dismiss this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants